Skip to content

fix:Keep subtitles visible during Next-up preview - #1150

Merged
PartyDonut merged 3 commits into
DonutWare:developfrom
Haryk96:fix/keep-subtitles-visible-in-next-up
Oct 25, 2026
Merged

PartyDonut merged 3 commits into
DonutWare:developfrom
Haryk96:fix/keep-subtitles-visible-in-next-up

Conversation

@Haryk96

@Haryk96 Haryk96 commented Sep 23, 2026

Copy link
Copy Markdown
Contributor

Pull Request Description

Keep subtitles visible when the Next-up split screen appears. The player now fades the controls while leaving the existing subtitle widget visible over the reduced video.

Issue Being Fixed

Near the end of an episode, Next-up faded the controls layer. That layer also contained subtitles, so dialogue continued playing without visible captions.

Screenshots / Recordings

Before

Screenshot 2026-09-23 at 21 10 37

After

Screenshot 2026-09-23 at 21 25 55

Tested On

  • Android
  • Android TV
  • iOS
  • Linux
  • Windows
  • macOS — visually checked the Next-up scene
  • Web

Checklist

  • No new package was added.
  • Changes are related to this issue.

@Haryk96
Haryk96 marked this pull request as ready for review September 23, 2026 19:41
@Haryk96 Haryk96 changed the title Keep subtitles visible during Next-up preview fix:Keep subtitles visible during Next-up preview Sep 23, 2026
@Haryk96

Haryk96 commented Sep 29, 2026

Copy link
Copy Markdown
Contributor Author

@PartyDonut can you please take a look? Thanks a lot 😊

PartyDonut
PartyDonut previously approved these changes Oct 25, 2026

@PartyDonut PartyDonut left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, thanks!

# Conflicts:
#	lib/screens/video_player/video_player_controls.dart
@PartyDonut PartyDonut added the bug Something isn't working label Oct 25, 2026
@PartyDonut
PartyDonut merged commit 6217b49 into DonutWare:develop Oct 25, 2026
1 check passed
@github-project-automation github-project-automation Bot moved this to Done in Fladder Oct 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants